Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ZDTM test assorted improvements #2198

Merged
merged 1 commit into from
Oct 13, 2023

Conversation

osctobe
Copy link
Contributor

@osctobe osctobe commented Jun 15, 2023

This series contains a mostly-independent patches that improve test debuggability and make running the tests easier to parallelize.

test/zdtm.py Fixed Show fixed Hide fixed
@adrianreber
Copy link
Member

It seems the Python import changes are breaking lot's of CI runs.

@osctobe
Copy link
Contributor Author

osctobe commented Jun 16, 2023

It seems the Python import changes are breaking lot's of CI runs.

Dropped the patch for now.

test/zdtm.py Outdated Show resolved Hide resolved
@codecov-commenter
Copy link

codecov-commenter commented Jun 19, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (42c1c84) 70.49% compared to head (ef85e66) 70.60%.

❗ Current head ef85e66 differs from pull request most recent head b9fd4a4. Consider uploading reports for the commit b9fd4a4 to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##           criu-dev    #2198      +/-   ##
============================================
+ Coverage     70.49%   70.60%   +0.10%     
============================================
  Files           132      134       +2     
  Lines         33509    33516       +7     
============================================
+ Hits          23623    23663      +40     
+ Misses         9886     9853      -33     

see 13 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

test/zdtm.py Outdated Show resolved Hide resolved
test/zdtm.py Outdated Show resolved Hide resolved
@osctobe osctobe force-pushed the zdtm-tests-1 branch 3 times, most recently from 1e19797 to 8f7a139 Compare June 26, 2023 21:01
test/zdtm.py Outdated Show resolved Hide resolved
test/zdtm.py Outdated Show resolved Hide resolved
@osctobe osctobe force-pushed the zdtm-tests-1 branch 3 times, most recently from fe23dd1 to 571cb01 Compare August 2, 2023 11:55
@osctobe osctobe force-pushed the zdtm-tests-1 branch 3 times, most recently from c068566 to cbd48bc Compare August 9, 2023 17:16
@osctobe osctobe requested a review from avagin August 9, 2023 17:16
@osctobe osctobe force-pushed the zdtm-tests-1 branch 2 times, most recently from 9d1d2e9 to 925cbc4 Compare August 21, 2023 15:00
@osctobe osctobe requested a review from avagin August 22, 2023 09:53
@osctobe osctobe force-pushed the zdtm-tests-1 branch 4 times, most recently from c1718c7 to 0a68122 Compare August 28, 2023 12:06
@osctobe osctobe force-pushed the zdtm-tests-1 branch 2 times, most recently from 7099e75 to 9d32c51 Compare August 30, 2023 08:46
cgroup_ifpriomap test needs net_prio cgroup, which might not be
available. Make the .checkskip script check it.

Signed-off-by: Michał Mirosław <[email protected]>
@avagin avagin merged commit 711775f into checkpoint-restore:criu-dev Oct 13, 2023
33 of 39 checks passed
@osctobe osctobe deleted the zdtm-tests-1 branch November 14, 2023 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants